Skip to content

fix(release): give the integration test path dependencies version requirements - #266

Merged
gregorydemay merged 1 commit into
mainfrom
ci/version-int-test-deps
Oct 9, 2026
Merged

gregorydemay merged 1 commit into
mainfrom
ci/version-int-test-deps

Conversation

@gregorydemay

Copy link
Copy Markdown
Contributor

Unblocks minter releases after the first one. Once a cksol_minter-v* tag exists, release-plz packages the workspace at that tag to compute the next version, and cargo package rejects integration_tests/Cargo.toml because its cksol-types* path dependencies have no version requirement. A local release-plz dry run with a baseline tag fails on main and succeeds with this change.

Since release-plz will now rewrite this manifest when cksol-types bumps, it is added back to the files the release bot may edit.

🤖 Generated with Claude Code

…uirements

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 9, 2026 11:02
@gregorydemay
gregorydemay marked this pull request as ready for review October 9, 2026 11:02
@gregorydemay
gregorydemay requested a review from a team as a code owner October 9, 2026 11:02
@zeropath-ai

zeropath-ai Bot commented Oct 9, 2026

Copy link
Copy Markdown

✅ No security or compliance issues detected. Reviewed everything up to 7ea4b4a.

Security Overview
Detected Code Changes
Change Type Relevant files
Enhancement ► integration_tests/Cargo.toml
    Update cksol-types and cksol-types-internal dependencies to use version 0.1.0 while keeping path references
► integration_tests/Cargo.toml
    csold as path-based dependencies updated to version 0.1.0 for both cksol-types and cksol-types-internal

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The dependency versions match their local packages and the bot allowlist supports future automated updates.

0 open findings

What changed in this PR

Adds version requirements to integration-test path dependencies so release-plz packaging succeeds after baseline tags exist.

Changes:

  • Adds matching 0.1.0 version requirements for both local types crates.
  • Allows release-plz to update the integration-test manifest.
File Description
integration_tests/​Cargo.toml Adds required versions to path dependencies.
.github/​repo_policies/​BOT_APPROVED_FILES Authorizes release bot manifest updates.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@gregorydemay
gregorydemay added this pull request to the merge queue Oct 9, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 9, 2026
@gregorydemay
gregorydemay added this pull request to the merge queue Oct 9, 2026
Merged via the queue into main with commit 46e56e6 Oct 9, 2026
24 checks passed
@gregorydemay
gregorydemay deleted the ci/version-int-test-deps branch October 9, 2026 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants